Skip to content

fix: let spm use base class for version #1199

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Sep 9, 2015
Merged

Conversation

satra
Copy link
Member

@satra satra commented Aug 31, 2015

temporary solution for #1195

@@ -766,7 +766,8 @@ class Segment(SPMCommand):

"""

if Info.version() and Info.version()['name'] == "SPM12":
_local_version = SPMCommand().version
if _local_version and _local_version['name'] == "SPM12":
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not

if _local_version and _local_version == "SPM12"

?

* upstream/master: (39 commits)
  fixed support for different folder structures in dicom importer
  fix: check if plugin args is None
  sty: autogenerate fix
  fix: make changes for release
  fix: deploy only nipy/nipype master branch
  fix for identifying if one or multiple durations are employed to describe the condition
  corrected line of modelgen and added test for variable concatenated events
  fix
  test
  FIX: Joint Fusion multimodal Interface Fix
  fix parse_inputs for interfaces not defining gradients
  add FSL bvec/bval format explicit to all interfaces
  fix doctests
  add tck2vtk
  add nohash=True on nthreads option
  add auto tests, fix doctests
  fix errors and missing parts
  add tck2connectome interface
  add interface for fs_parc_replace_sgm_first
  fix default file finding
  ...
chrisgorgo added a commit that referenced this pull request Sep 9, 2015
fix: let spm use base class for version
@chrisgorgo chrisgorgo merged commit 0f7597f into nipy:master Sep 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants